Hi @mdayican ,
My comments as below,

1. Remove unused method from src/main/java/uk/gov/hmcts/dm/repository/DocumentContentVersionRepository.java - markMimeTypeUpdated()
2. We have 12 attributes in DocumentContentVersion. We are updating 2 attributes - mimeType & mimeTypeUpdated. Instead of making the DB call to load the entire object. We can use something like below which I believe is efficient and lots of lines of code can be removed related to loading and try/catch.

//Efficient: repository-level update to avoid loading the entity
// In DocumentContentVersionRepository:
@Modifying
@Query("update DocumentContentVersion d set d.mimeType = :mimeType, d.mimeTypeUpdated = true where d.id = :id")
int updateMimeType(@Param("id") UUID id, @Param("mimeType") String mimeType);

Hi @mdayican , My comments as below,

1. Remove unused method from src/main/java/uk/gov/hmcts/dm/repository/DocumentContentVersionRepository.java - markMimeTypeUpdated()
2. We have 12 attributes in DocumentContentVersion. We are updating 2 attributes - mimeType & mimeTypeUpdated. Instead of making the DB call to load the entire object. We can use something like below which I believe is efficient and lots of lines of code can be removed related to loading and try/catch.

//Efficient: repository-level update to avoid loading the entity
// In DocumentContentVersionRepository:
@Modifying
@Query("update DocumentContentVersion d set d.mimeType = :mimeType, d.mimeTypeUpdated = true where d.id = :id")
int updateMimeType(@Param("id") UUID id, @Param("mimeType") String mimeType);

fixed.